Skip to content

proper reporting of rust version based on git commit hash - build-info#123

Merged
stephen-derosa merged 3 commits into
livekit:mainfrom
stephen-derosa:sderosa/BOT-348-ffi-lib-reporting
May 13, 2026
Merged

proper reporting of rust version based on git commit hash - build-info#123
stephen-derosa merged 3 commits into
livekit:mainfrom
stephen-derosa:sderosa/BOT-348-ffi-lib-reporting

Conversation

@stephen-derosa
Copy link
Copy Markdown
Collaborator

@stephen-derosa stephen-derosa commented May 11, 2026

Overview

Fixes C++ SDK version reporting by sending the release version (LIVEKIT_BUILD_VERSION) to the Rust FFI instead of the build-flavored value (LIVEKIT_BUILD_VERSION_FULL, e.g. 1.2.3-release).

Also adds release provenance metadata to SDK bundles via share/livekit/build-info.json, recording the C++ SDK version, C++ commit, Rust submodule commit, and bundled livekit-ffi crate version. Release CI now validates that bundled artifacts include the expected version metadata and do not fall back to 0.0.0.

Validation:

Added a C++ unit test to ensure the server-facing version does not include -debug / -release.

Verified locally with ./build.sh release --version 1.2.3-rc1 --bundle --prefix sdk-out/build-info-smoke. The installed bundle contains share/livekit/build-info.json, preserves the full prerelease SDK version, emits UTC ISO-8601 build_date, and matches include/livekit/build.h.

BOT-348

Related MR

@stephen-derosa stephen-derosa self-assigned this May 11, 2026
@stephen-derosa stephen-derosa force-pushed the sderosa/BOT-348-ffi-lib-reporting branch 2 times, most recently from 7de6a21 to 93c8e84 Compare May 12, 2026 22:28
Comment thread .build-info.json.in
Comment thread CMakeLists.txt
@stephen-derosa stephen-derosa force-pushed the sderosa/BOT-348-ffi-lib-reporting branch from 9d7abc5 to cb091f6 Compare May 12, 2026 22:55
Copy link
Copy Markdown
Collaborator

@xianshijing-lk xianshijing-lk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

some comments

Comment thread .build-info.json.in
Comment thread CMakeLists.txt Outdated
@stephen-derosa stephen-derosa force-pushed the sderosa/BOT-348-ffi-lib-reporting branch from cb091f6 to b44929b Compare May 13, 2026 15:53
@stephen-derosa stephen-derosa merged commit 49b0077 into livekit:main May 13, 2026
31 of 33 checks passed
@stephen-derosa stephen-derosa deleted the sderosa/BOT-348-ffi-lib-reporting branch May 13, 2026 20:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants